Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Feature][core] support read arrow data #8137

Merged
merged 1 commit into from
Dec 6, 2024
Merged

Conversation

hawk9821
Copy link
Contributor

@hawk9821 hawk9821 commented Nov 25, 2024

Purpose of this pull request

support read arrow data, convert arrow data to SeatunnelRow. At present, doris connector and starrocks connector use arrow mode to read data and extract this part of logic.

Does this PR introduce any user-facing change?

doris connector source , starrocks connector source

How was this patch tested?

UT: Arrow2SeatunnelRowReaderTest#testSeatunnelRow
Arrow2SeatunnelRowReaderTest#testConvertArrowSpeed

Check list

@github-actions github-actions bot added dependencies Pull requests that update a dependency file connectors-v2 api labels Nov 25, 2024
@github-actions github-actions bot removed the api label Nov 26, 2024
@hawk9821 hawk9821 force-pushed the arrow branch 3 times, most recently from ccd1b1b to 7fe4021 Compare November 28, 2024 14:10
@github-actions github-actions bot added the api label Dec 2, 2024
@hawk9821 hawk9821 force-pushed the arrow branch 2 times, most recently from 078e298 to 96ab87e Compare December 2, 2024 09:32
@hawk9821 hawk9821 force-pushed the arrow branch 4 times, most recently from 890ef6f to e56e7cb Compare December 4, 2024 00:35
Copy link
Member

@hailin0 hailin0 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@Hisoka-X Hisoka-X merged commit 4710ea0 into apache:dev Dec 6, 2024
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api approved connectors-v2 dependencies Pull requests that update a dependency file reviewed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants